Skip to content

Fix flakey tests. #5014

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 5 commits into from
Closed

Fix flakey tests. #5014

wants to merge 5 commits into from

Conversation

christophstrobl
Copy link
Member

Use vector-serach test tag to indicate potential slow environment.

@christophstrobl christophstrobl marked this pull request as ready for review July 7, 2025 10:50
@christophstrobl christophstrobl mentioned this pull request Jul 7, 2025
Open
christophstrobl and others added 2 commits July 7, 2025 14:56
@christophstrobl @mp911de
Introduce @EnableIfVectorSearchAvailable to wait and conditionally …
02687b8 
…skip tests.

We now wait until a search index becomes available. If the search index doesn't come alive within 60 seconds, we skip that test (or test class).

Closes: #5013
Original pull request: #5014
@mp911de
Polishing.
209269c 
Simplify tests by using JUnit's meta-annotation pickup of ExtendWith. Hide extensions.

See: #5013
Original pull request: #5014
@mp911de mp911de added this to the 4.5.2 (2025.0.2) milestone Jul 8, 2025
@mp911de mp911de added the type: task A general task label Jul 8, 2025
@mp911de mp911de linked an issue Jul 8, 2025 that may be closed by this pull request
Fix flakey vector search tests #5013
Closed
mp911de pushed a commit that referenced this pull request Jul 8, 2025
@christophstrobl @mp911de
Introduce @EnableIfVectorSearchAvailable to wait and conditionally …
9e578a1 
…skip tests.

We now wait until a search index becomes available. If the search index doesn't come alive within 60 seconds, we skip that test (or test class).

Closes: #5013
Original pull request: #5014
mp911de added a commit that referenced this pull request Jul 8, 2025
@mp911de
Polishing.
9c21cb0 
Simplify tests by using JUnit's meta-annotation pickup of ExtendWith. Hide extensions.

See: #5013
Original pull request: #5014
mp911de pushed a commit that referenced this pull request Jul 8, 2025
@christophstrobl @mp911de
Introduce @EnableIfVectorSearchAvailable to wait and conditionally …
d43d6b0 
…skip tests.

We now wait until a search index becomes available. If the search index doesn't come alive within 60 seconds, we skip that test (or test class).

Closes: #5013
Original pull request: #5014
mp911de added a commit that referenced this pull request Jul 8, 2025
@mp911de
Polishing.
925e3cd 
Simplify tests by using JUnit's meta-annotation pickup of ExtendWith. Hide extensions.

See: #5013
Original pull request: #5014
@mp911de mp911de closed this Jul 8, 2025
@mp911de mp911de deleted the issue/5013 branch July 8, 2025 10:32
mp911de added a commit that referenced this pull request Jul 8, 2025
@mp911de
Polishing.
e22a0d2 
Consider listen port in addition to shell command.

See: #5013
Original pull request: #5014
mp911de added a commit that referenced this pull request Jul 8, 2025
@mp911de
Polishing.
859ce08 
Consider listen port in addition to shell command.

See: #5013
Original pull request: #5014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@mp911de mp911de

Labels
type: task A general task
Projects
None yet
Milestone
4.5.2 (2025.0.2)
Development

Successfully merging this pull request may close these issues.

Fix flakey vector search tests
2 participants
@christophstrobl @mp911de